Skip to content

Conversation

romanowska
Copy link
Contributor

@romanowska romanowska commented Oct 16, 2025

πŸ“ Description

  • Fix members component test
  • Fix members e2e tests
  • Fix minor bugs

✨ Type of Change

Select the type of change your PR introduces:

  • 🐞 Bug fix – Non-breaking change which fixes an issue
  • πŸš€ New feature – Non-breaking change which adds functionality
  • πŸ”¨ Refactor – Non-breaking change which refactors the code base
  • πŸ’₯ Breaking change – Changes that break existing functionality
  • πŸ“š Documentation update
  • πŸ”’ Security update
  • πŸ§ͺ Tests

πŸ§ͺ Testing Scenarios

Describe how the changes were tested and how reviewers can test them too:

  • βœ… Tested manually
  • πŸ€– Run automated end-to-end tests

βœ… Checklist

Before submitting the PR, ensure the following:

  • πŸ” PR title is clear and meaningful
  • ✍️ PR description clearly explains the changes and their reason
  • πŸ“ I have linked the PR to the corresponding GitHub Issues, if any
  • πŸ’¬ I have commented my code, especially in hard-to-understand areas
  • πŸ“„ I have made corresponding changes to the documentation
  • βœ… I have added tests that prove my fix is effective or my feature works

@Copilot Copilot AI review requested due to automatic review settings October 16, 2025 06:48
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR temporarily disables a failing end-to-end test in the daily test suite by skipping the "Members management" test. The change includes a TODO comment indicating that a proper fix will be implemented in a separate PR.

  • Adds test.skip() to temporarily disable the failing Members management test
  • Includes a TODO comment explaining the temporary nature of the skip

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions github-actions bot added the UI label Oct 16, 2025
@romanowska romanowska changed the title Temporarily skip members test on daily Fix members component and e2e tests Oct 22, 2025
Copy link
Contributor

@ActiveChooN ActiveChooN left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few minor comments, but it's generally LGTM

password: 'Test1234',
role: USER_ROLE.WORKSPACE_ADMIN,
workspaceRole: USER_ROLE.WORKSPACE_ADMIN,
organizationRole: USER_ROLE.ORGANIZATION_ADMIN,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The user should not have the ORGNIZATION_ADMIN role and any other role simultaneously.

await membersPage.resetSearchFilter();
});

await test.step('Edits workspace admin and workspace contributor member', async () => {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this test and the next one be renamed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants